Skip to content

Remove fantom test specific methods from the C++ API snapshot (#56247)#56247

Closed
coado wants to merge 5 commits into
facebook:mainfrom
coado:export-D97466201
Closed

Remove fantom test specific methods from the C++ API snapshot (#56247)#56247
coado wants to merge 5 commits into
facebook:mainfrom
coado:export-D97466201

Conversation

@coado
Copy link
Copy Markdown
Contributor

@coado coado commented Mar 27, 2026

Summary:

Removes fantom test specific methods, namely:

  • NativeFantomTestSpecificMethods,
  • FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 27, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner labels Mar 27, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 27, 2026

@coado has exported this pull request. If you are a Meta employee, you can view the originating Diff in D97466201.

Copy link
Copy Markdown
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot changed the title Remove fantom test specific methods from the C++ API snapshot Remove fantom test specific methods from the C++ API snapshot (#56247) Mar 27, 2026
@coado coado force-pushed the export-D97466201 branch from e7e6827 to b8108fa Compare March 27, 2026 11:09
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
…ok#56247)

Summary:

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
coado added a commit to coado/react-native that referenced this pull request Mar 27, 2026
…ok#56247)

Summary:
Pull Request resolved: facebook#56247

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
@coado coado force-pushed the export-D97466201 branch from b8108fa to eb9c7ce Compare March 27, 2026 11:13
Dawid Małecki added 4 commits March 31, 2026 04:30
Summary:
This diff introduces initial snapshots of the Cxx public API in a form of a couple .api files, namely:

- ReactAndroidDebugCxx.api
- ReactAndroidReleaseCxx.api
- ReactAppleDebugCxx.api
- ReactAppleReleaseCxx.api
- ReactCommonDebugCxx.api
- ReactCommonReleaseCxx.api

that is for `ReactAndroid`, `ReactApple`, and shared `ReactCommon` in two variants `Release` and `Debug` each.

Differential Revision: D97296573
Summary:
Pull Request resolved: facebook#56042

Adds CI workflow for validating whether the current C++ API snapshot is equivalent with the generated one.

Changelog:
[Internal]

Differential Revision: D95963515
Summary:
Removes leaked test code from the snapshot.

Changelog:
[Internal]

Differential Revision: D97457090
Summary:
Refactor and add global exclude patterns to the C++ stable API snapshot config. This enables better modularization of shared exclude patterns coming from the assumption that ReactAndroid and ReactApple consists of ReactCommon in their inputs and some paths should not be processed on each platform.

Changelog:
[Internal]

Differential Revision: D97465431
@coado coado force-pushed the export-D97466201 branch from eb9c7ce to 7051b84 Compare March 31, 2026 11:33
coado added a commit to coado/react-native that referenced this pull request Mar 31, 2026
…ok#56247)

Summary:

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
…ok#56247)

Summary:
Pull Request resolved: facebook#56247

Removes fantom test specific methods, namely:
- NativeFantomTestSpecificMethods,
- FantomForcedCloneCommitHook

from the C++ API snapshot.

Changelog:
[Internal]

Reviewed By: cortinico

Differential Revision: D97466201
@coado coado force-pushed the export-D97466201 branch from 7051b84 to 227ba13 Compare March 31, 2026 11:36
@meta-codesync meta-codesync Bot closed this in 868eb6c Mar 31, 2026
@facebook-github-tools facebook-github-tools Bot added the Merged This PR has been merged. label Mar 31, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Mar 31, 2026

This pull request has been merged in 868eb6c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants